Skip to content

Conversation

@bjohansebas
Copy link
Contributor

Description (required)

Related issues & labels (optional)

  • Closes #
  • Suggested label:

@netlify
Copy link

netlify bot commented Oct 25, 2025

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 3f52b50
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/68fd1f0b7a5df50008801efc
😎 Deploy Preview https://deploy-preview-12629--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Oct 25, 2025
@astrobot-houston
Copy link
Contributor

astrobot-houston commented Oct 25, 2025

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
es/tutorial/0-introduction/index.mdx Localization changed, will be marked as complete.
es/tutorial/1-setup/1.mdx Localization changed, will be marked as complete.
es/tutorial/1-setup/2.mdx Localization changed, will be marked as complete.
es/tutorial/1-setup/5.mdx Localization changed, will be marked as complete.
es/tutorial/1-setup/index.mdx Localization changed, will be marked as complete.
es/tutorial/2-pages/2.mdx Localization changed, will be marked as complete.
es/tutorial/2-pages/4.mdx Localization changed, will be marked as complete.
es/tutorial/2-pages/5.mdx Localization changed, will be marked as complete.
es/tutorial/2-pages/index.mdx Localization changed, will be marked as complete.
es/tutorial/3-components/1.mdx Localization changed, will be marked as complete.
es/tutorial/3-components/3.mdx Localization changed, will be marked as complete.
es/tutorial/3-components/4.mdx Localization changed, will be marked as complete.
es/tutorial/3-components/index.mdx Localization changed, will be marked as complete.
es/tutorial/4-layouts/1.mdx Localization changed, will be marked as complete.
es/tutorial/4-layouts/2.mdx Localization changed, will be marked as complete.
es/tutorial/4-layouts/3.mdx Localization changed, will be marked as complete.
es/tutorial/4-layouts/index.mdx Localization changed, will be marked as complete.
es/tutorial/5-astro-api/1.mdx Localization changed, will be marked as complete.
es/tutorial/5-astro-api/2.mdx Localization changed, will be marked as complete.
es/tutorial/5-astro-api/3.mdx Localization changed, will be marked as complete.
es/tutorial/5-astro-api/index.mdx Localization changed, will be marked as complete.
es/tutorial/6-islands/2.mdx Localization changed, will be marked as complete.
es/tutorial/6-islands/3.mdx Localization changed, will be marked as complete.
es/tutorial/6-islands/4.mdx Localization changed, will be marked as complete.
es/tutorial/6-islands/index.mdx Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Signed-off-by: Sebastian Beltran <[email protected]>
@bjohansebas bjohansebas marked this pull request as draft October 25, 2025 16:34
Signed-off-by: Sebastian Beltran <[email protected]>
Signed-off-by: Sebastian Beltran <[email protected]>
Signed-off-by: Sebastian Beltran <[email protected]>
Signed-off-by: Sebastian Beltran <[email protected]>
Signed-off-by: Sebastian Beltran <[email protected]>
Signed-off-by: Sebastian Beltran <[email protected]>
@bjohansebas bjohansebas marked this pull request as ready for review October 25, 2025 19:00
Signed-off-by: Sebastian Beltran <[email protected]>
Copy link
Contributor

@SergioDiez SergioDiez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just added 3 minor issues. Amazing work!

</Fragment>
</PackageManagerTabs>

2. Escribe `y` para instalar `create-astro`.
Copy link
Contributor

@SergioDiez SergioDiez Nov 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This 2. point is duplicated in the next line.
This one looks better, so I would delete the L52.


<Steps>
1. Actualiza `Navigation.astro` con la clase CSS para controlar tus enlaces de navegación. Envuelve los enlaces de navegación existentes en un `<div>` con la clase `nav-links`.
1. Actualiza `Navigation.astro` con la clase CSS para controlar tus enlaces de navegación. Envuelve los enlaces de navegación existentes en un `<div>` con la clase `nav-links` y el atributo `id` establecido como `main-menu`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The id is not formatted as code in the English docs. I think it's better like you wrote it, but I don't know if it should be kept as English docs for consistency.

1. Update `Navigation.astro` with the CSS class to control your navigation links. Wrap the existing navigation links in a `<div>` with the class `nav-links` and the id attribute set to `main-menu`.

Comment on lines 81 to +82
]
}
};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The semicolon should be placed in the array closure

Suggested change
]
}
};
];
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants